Skip to content

Conversation

@ethDreamer
Copy link
Member

Issue Addressed

This is the second PR in the conga line! 🎉

shane-moore and others added 25 commits August 26, 2025 09:46
* use builder_pending_payments_limit in upgrade gloas

* check_all_blocks_from_altair_to_fulu test to not cover gloas for now

* store_tests fixes

* remove gloas fork from CI network testing for now

* remove gloas fork from CI network testing for now
* gloas envelope helpers

* rename execution_bid to execution_payload_bid

* builder_payment_threshold_numerator and builder_payment_threshold_denominator are not configurable but are constants

* update payload attestation message signature to be unaggregated type

* updates per pr review
shane-moore and others added 8 commits December 1, 2025 15:13
* add process_builder_pending_payments to epoch processing

* per_slot_processing modified

* add process_builder_pending_payments to single pass process epoch

* update process_builder_pending_payments per latest spec
Co-authored-by: ethDreamer <mark@sigmaprime.io>
…ble-2

Merge `unstable` into `gloas-containers` (FIXED)
Base automatically changed from gloas-containers to unstable December 16, 2025 06:45
@mergify mergify bot requested a review from jxs as a code owner December 16, 2025 06:45
@michaelsproul
Copy link
Member

TODO:

  • Undo Michael's dodgy assumption around builder indices being validator indices (see 9e365e5).
  • Refactor withdrawals code to match spec (remove large blocks that are inlined)
  • Introduce concept of state.builders (we don't have this in BeaconState yet)
  • Possibly add another ValidatorPubkeyCache for the builder pubkeys

/// Cache for deduplicating `notify_new_payload` requests.
///
/// Handles both in-flight requests and recently completed requests.
new_payload_cache: Option<NewPayloadCache>,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this cache necessary?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants